Skip to content

Fix RelWithDebInfo to be release with debug info#11211

Merged
cmcfarlen merged 2 commits intoapache:masterfrom
cmcfarlen:better-relwithdebinfo
May 13, 2024
Merged

Fix RelWithDebInfo to be release with debug info#11211
cmcfarlen merged 2 commits intoapache:masterfrom
cmcfarlen:better-relwithdebinfo

Conversation

@cmcfarlen
Copy link
Copy Markdown
Contributor

No description provided.

@cmcfarlen cmcfarlen added the CMake work related to CMakes scripts or issues label Apr 4, 2024
@cmcfarlen cmcfarlen requested a review from JosiahWI April 4, 2024 12:55
@cmcfarlen cmcfarlen self-assigned this Apr 4, 2024
@cmcfarlen cmcfarlen added this to the 10.1.0 milestone Apr 4, 2024
Copy link
Copy Markdown
Contributor

@JosiahWI JosiahWI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good, but could we add a brief comment explaining this was done for consistency between build types?

@cmcfarlen cmcfarlen force-pushed the better-relwithdebinfo branch from 313d5e6 to 81d9733 Compare April 5, 2024 14:58
@cmcfarlen cmcfarlen requested a review from JosiahWI April 5, 2024 14:58
Copy link
Copy Markdown
Contributor

@JosiahWI JosiahWI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks, looks good.

@bryancall bryancall requested review from dragon512 and zwoop April 8, 2024 22:21
@bryancall
Copy link
Copy Markdown
Contributor

@dragon512 and @zwoop We talked about this in the PR/issue scrub and would like to have your input. No one felt strongly about having this in or not.

@cmcfarlen cmcfarlen merged commit a7a150d into apache:master May 13, 2024
@cmcfarlen cmcfarlen deleted the better-relwithdebinfo branch May 13, 2024 22:50
@cmcfarlen cmcfarlen modified the milestones: 10.1.0, 10.0.0 May 16, 2024
cmcfarlen added a commit that referenced this pull request May 16, 2024
* Fix RelWithDebInfo to be release with debug info

* Add comment

(cherry picked from commit a7a150d)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

CMake work related to CMakes scripts or issues

Projects

Status: picked-10.0.0

Development

Successfully merging this pull request may close these issues.

3 participants